Skip to content

[To dev/1.3] Fix async recover TsfileResource error handle#14235

Merged
HTHou merged 3 commits intodev/1.3from
fix_async_recover_error_1.3
Nov 29, 2024
Merged

[To dev/1.3] Fix async recover TsfileResource error handle#14235
HTHou merged 3 commits intodev/1.3from
fix_async_recover_error_1.3

Conversation

@HTHou
Copy link
Copy Markdown
Contributor

@HTHou HTHou commented Nov 28, 2024

Description

cherry-pick #14234

@HTHou HTHou changed the title Fix async recover TsfileResource error handle [To dev/1.3] Fix async recover TsfileResource error handle Nov 28, 2024
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 28, 2024

Codecov Report

Attention: Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.

Project coverage is 41.20%. Comparing base (57b1309) to head (64ed830).
Report is 1 commits behind head on dev/1.3.

Files with missing lines Patch % Lines
.../iotdb/db/storageengine/dataregion/DataRegion.java 50.00% 1 Missing ⚠️
...geengine/dataregion/utils/TsFileResourceUtils.java 85.71% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev/1.3   #14235      +/-   ##
=============================================
- Coverage      41.21%   41.20%   -0.01%     
  Complexity        71       71              
=============================================
  Files           3470     3470              
  Lines         222242   222247       +5     
  Branches       26403    26405       +2     
=============================================
- Hits           91587    91579       -8     
- Misses        130655   130668      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HTHou HTHou merged commit d80e290 into dev/1.3 Nov 29, 2024
@HTHou HTHou deleted the fix_async_recover_error_1.3 branch November 29, 2024 01:32
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants